Reviewing files that changed from the base of the PR and between c18b823 and 3b9267f.

• pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

• .vscode/settings.json (1 hunks)
• apps/browser-extension/package.json (0 hunks)
• apps/browser-extension/tsconfig.json (1 hunks)
• apps/browser-extension/vite-plugins.ts (1 hunks)
• apps/browser-extension/vite.config.chrome.ts (1 hunks)
• apps/browser-extension/vite.config.firefox.ts (1 hunks)
• apps/discord-bot/package.json (1 hunks)
• apps/discord-bot/tsconfig.json (1 hunks)
• apps/website/package.json (0 hunks)
• apps/website/tsconfig.json (1 hunks)
• biome.json (1 hunks)
• package.json (1 hunks)
• packages/components/package.json (1 hunks)
• packages/components/src/carousel.tsx (1 hunks)
• packages/components/src/form.tsx (1 hunks)
• packages/components/tsconfig.json (1 hunks)
• packages/components/tsup.config.ts (0 hunks)
• packages/engine/package.json (1 hunks)
• packages/engine/src/execute/fetch.ts (1 hunks)
• packages/engine/src/execute/index.ts (1 hunks)
• packages/engine/src/runtimes/fetch.ts (1 hunks)
• packages/engine/src/runtimes/getters.ts (1 hunks)
• packages/engine/src/runtimes/index.ts (1 hunks)
• packages/engine/tsconfig.json (1 hunks)
• packages/engine/tsup.config.ts (0 hunks)
• packages/helpers/package.json (1 hunks)
• packages/helpers/src/better-fetch/better-fetch.ts (1 hunks)
• packages/helpers/src/better-fetch/index.ts (1 hunks)
• packages/helpers/src/create-logger/create-logger.ts (1 hunks)
• packages/helpers/src/create-logger/index.ts (1 hunks)
• packages/helpers/tsconfig.json (1 hunks)
• packages/helpers/tsup.config.ts (0 hunks)
• packages/hooks/package.json (1 hunks)
• packages/hooks/src/event-callback.ts (1 hunks)
• packages/hooks/src/event-listener.ts (1 hunks)
• packages/hooks/src/media-query.ts (1 hunks)
• packages/hooks/tsup.config.ts (0 hunks)
• packages/shapes/package.json (1 hunks)
• packages/shapes/src/index.ts (1 hunks)
• packages/shapes/src/runtime.ts (1 hunks)
• packages/shapes/tsconfig.json (1 hunks)
• packages/shapes/tsup.config.ts (0 hunks)
• tsconfig.base.json (1 hunks)
• tsconfig.react.json (1 hunks)

• apps/website/package.json
• packages/helpers/tsup.config.ts
• packages/shapes/tsup.config.ts
• packages/engine/tsup.config.ts
• packages/hooks/tsup.config.ts
• packages/components/tsup.config.ts
• apps/browser-extension/package.json

apps/discord-bot/tsconfig.json (1)

1-2: Minimal tsconfig: Validate Extended Configuration

The tsconfig for the discord-bot app has been pared down to only extend "../../tsconfig.base.json". Please verify that all necessary compiler options—as well as file inclusion/exclusion rules—are properly defined in the base config so that the project compiles as expected.

packages/components/src/form.tsx (1)

14-14: Explicit File Extension in Import

The import for the Label component now explicitly references ./label.jsx, which improves clarity regarding the file type and aligns with the new module resolution strategy. Please confirm that the file’s naming and extension are consistent across the codebase.

apps/discord-bot/package.json (2)

15-17: Review Build Script Command

The "build" script has been updated to "pnpm tsup". Considering the PR objective to replace tsup with tsc, please verify that this command is intentionally retained for the Discord bot or if it should be updated to use tsc to ensure consistency across projects.

---

24-27: Dependency Reintroduction Confirmation

The es-toolkit package is reintroduced with version "^1.32.0". Confirm that this version is compatible with the current codebase and that its reintroduction was intentional given the overall dependency and build process adjustments.

packages/hooks/src/media-query.ts (1)

2-2: Explicit Module Resolution for Hook

The import statement for useIsomorphicLayoutEffect now includes the .js extension. This explicit declaration improves module resolution clarity and is consistent with the new configuration standards. Ensure this pattern is uniformly applied where necessary.

packages/helpers/src/create-logger/create-logger.ts (1)

1-1: Explicit File Extension in Logger Import

The update to import hexToAnsi from './hex-to-ansi.js' enhances clarity by explicitly stating the file type. This change aligns well with the new module specifier settings and should help prevent potential resolution issues.

packages/hooks/src/event-callback.ts (1)

2-2: Enforce Explicit Module Resolution with File Extension
Updating the import to include the .js extension helps ensure that module resolution works consistently in ESM environments. Verify that this change is applied uniformly across similar modules.

.vscode/settings.json (1)

30-30: Add TypeScript Import Specifier Preference Setting
The new setting "typescript.preferences.importModuleSpecifierEnding": "js" enforces explicit file extensions for module imports in TypeScript files, aligning with the updated module resolution strategy across the project.

packages/engine/tsconfig.json (1)

1-1: Utilize Base Configuration for Consistency
By eliminating the local compilerOptions section, this configuration now entirely relies on the base settings defined in ../../tsconfig.base.json. Confirm that this does not remove any critical options that the engine package might require.

packages/engine/src/execute/index.ts (1)

1-1: Update Export Statement to Use Explicit Extension
Changing the export statement to export * from './fetch.js'; aligns with the project's new module resolution strategy by explicitly specifying the file extension. Ensure that the generated JavaScript file fetch.js exists or is properly produced as part of the build process.

packages/shapes/src/runtime.ts (1)

2-2: Explicit File Extension in Import Statement
The update now explicitly includes the .js extension in the import (import { ExecuteOptions } from './execute.js';). This change promotes consistency with the new module resolution strategy and avoids ambiguity during TypeScript compilation (especially relevant since you're transitioning from tsup to tsc).

packages/helpers/src/better-fetch/index.ts (1)

1-1: Explicit Export with File Extension
The export statement has been updated to explicitly reference './better-fetch.js'. This helps ensure that module resolution is unambiguous under the updated TypeScript setup and aligns with the overall changes to module specifiers across the project.

packages/helpers/src/create-logger/index.ts (1)

1-1: Updated Export to Include File Extension
Changing the export to export * from './create-logger.js'; clearly specifies the file to be imported at runtime. This explicit mode complements the new TS configuration and removes potential resolution issues.

packages/components/src/carousel.tsx (1)

9-9: Explicit Import File Extension for Button Component
The Button component import has been modified to include the .jsx extension (import { Button } from './button.jsx';). This adjustment is in line with the project-wide push for explicit file extensions, ensuring reliable module resolution under TSC.

packages/shapes/tsconfig.json (1)

1-1: Simplify and Centralize TypeScript Configuration
The removal of the local compilerOptions section—leaving only the "extends": "../../tsconfig.base.json"—simplifies this package’s TS configuration and centralizes configuration management. Please verify that all previously needed compiler options are now appropriately defined in the base configuration, to prevent any unintended side effects during compilation with tsc.

tsconfig.react.json (1)

1-7: New React TypeScript Configuration File Added

This file correctly extends the base configuration and sets "jsx": "preserve", ensuring that JSX is not transformed. It centralizes React-specific TypeScript settings for consistent use across projects.

packages/hooks/package.json (1)

10-12: Updated Build Script to Use tsc

The build script has been changed from using tsup to directly invoking tsc, which aligns perfectly with the PR objective to replace tsup with tsc and simplifies the build process.

biome.json (1)

2-3: Simplified Ignore List Configuration

The "files" section now only ignores "node_modules/", "dist/", and ".next/*", which streamlines the configuration. This reduces complexity by filtering out only the essential directories.

packages/shapes/package.json (1)

14-16: Adjusted Build Script for Shapes Package

The build script is updated to "build": "tsc", replacing the previous tsup invocation. This change matches the project-wide shift to TypeScript’s compiler for the build process.

packages/helpers/src/better-fetch/better-fetch.ts (1)

1-1: Explicit Import Extension for Module Resolution

The updated import with the explicit .js extension improves module resolution in ESM environments and maintains consistency with the broader update across the codebase.

packages/components/tsconfig.json (1)

1-1: Good switch to React-specific configuration.

Updating the configuration to extend from tsconfig.react.json is appropriate for a React components package. This change aligns with the PR's objective of replacing tsup with tsc and ensures that the components package has the correct TypeScript settings for React development.

packages/hooks/src/event-listener.ts (1)

2-2: Proper addition of the .js extension for ESM compatibility.

Adding the .js extension to the import path is necessary for ESM compatibility, as ES modules require explicit file extensions in import statements. This change aligns with the TypeScript setting "typescript.preferences.importModuleSpecifierEnding": "js" mentioned in the PR summary.

apps/browser-extension/vite-plugins.ts (1)

47-58:

Details

❓ Verification inconclusive

Good addition of plugin to handle import attributes.

The removeImportAttributes plugin is a useful addition for stripping the newer JavaScript with import attributes syntax, which might not be supported in all browser environments. This is particularly important for browser extensions where compatibility is crucial.

The regex pattern looks functional for most common cases, but consider testing with various import statement formats to ensure it handles all edge cases correctly.

---

🏁 Script executed:

#!/bin/bash
# Test the regex pattern against various import statement formats

# Create a temporary file with different import statement patterns
cat << EOF > test_imports.js
import foo from 'bar';
import foo from 'bar' with { type: 'json' };
import foo from "bar" with { type: 'json' };
import foo from "bar" with { type: "json" };
import { foo } from 'bar' with { type: 'json' };
import { foo, bar } from 'bar' with { type: 'json', assert: 'value' };
// Comment before import
import foo from 'bar' with { type: 'json' }; // Comment after import
  import foo from 'bar' with { type: 'json' };  // With whitespace
import foo from 'bar' /*comment*/ with { type: 'json' };
EOF

# Apply the regex transform manually
echo "Testing regex transformation results:"
echo "======================================"
cat test_imports.js | sed -E 's/(from\s['\"](?:.*?)['\"])(\swith\s\{.*?\})/\1/g'
echo "======================================"

# Clean up
rm test_imports.js

Length of output: 441

---

Manual Verification Needed: Import Attribute Plugin Regex

Nice addition of the plugin for stripping the newer JavaScript with import attributes. The regex appears to work for common cases, but further testing is required to ensure it correctly handles all edge cases—including varied quoting, whitespace, and inline comments.

• The provided test script produced a syntax error due to quoting issues with the sed command. Consider adjusting the sed command (e.g., using different delimiters or proper escaping) when testing the regex.
• Please manually verify the regex with additional examples to confirm it handles formats like inline comments, extra whitespace, and mixed quoting.

apps/browser-extension/vite.config.chrome.ts (2)

8-11: Properly structured imports with appropriate naming.

The imports are well-organized, using a named import structure that clearly shows the two plugin functions being imported from the vite-plugins file.

---

16-16: Correct integration of the new import attributes plugin.

The removeImportAttributes() plugin is appropriately placed in the plugins array, right after the existing removeExternalScriptLoading() plugin. Both plugins use enforce: 'pre' so this ordering is logical.

packages/engine/src/runtimes/index.ts (1)

1-2: Updated export paths to include .js extension

These changes are necessary for proper ESM compatibility when using tsc instead of tsup. Adding explicit .js extensions ensures correct module resolution in the compiled output.

apps/browser-extension/vite.config.firefox.ts (2)

8-11: Import structure updated to include the new removeImportAttributes plugin

Clean import structure that properly groups the related Vite plugins from the local file.

---

16-16: Added removeImportAttributes plugin to handle import attributes

This addition helps to remove with attributes from import statements, which is necessary after switching from tsup to tsc. The plugin placement in the array is appropriate, running early in the transformation chain.

packages/engine/package.json (1)

14-14:

Details

✅ Verification successful

Build command updated from tsup to tsc

This change aligns with the PR's goal of replacing tsup with tsc for the build process. The "check" script has been removed, which is consistent with the approach taken across the repository.

---

🏁 Script executed:

#!/bin/bash
# Verify if there are any tsconfig.json options that might need adjusting
# after the change from tsup to tsc

# Check for tsconfig.json in the same directory
if [ -f "packages/engine/tsconfig.json" ]; then
  echo "Found tsconfig.json in packages/engine, checking contents:"
  cat packages/engine/tsconfig.json
else
  echo "No tsconfig.json found in packages/engine, looking for parent configs:"
  find . -name "tsconfig.json" -type f -not -path "./node_modules/*" | xargs cat
fi

# Check if there was a tsup.config.ts file that had special options
find packages/engine -name "tsup.config.ts" -type f

Length of output: 299

---

Action Required: Verification Confirmed for Build Command Update

The build command change from tsup to tsc has been verified. The local tsconfig.json in packages/engine extends from ../../tsconfig.base.json, and no tsup.config.ts exists in the package, confirming that no special tsup-specific options need translation. The removal of the "check" script is consistent with the repository-wide approach.

packages/helpers/package.json (1)

16-16: Build command updated from tsup to tsc

The build script has been updated to use TypeScript's compiler directly instead of tsup, consistent with the PR's objective. This aligns with the changes made in other packages.

packages/engine/src/execute/fetch.ts (1)

8-10: Explicit Import Extensions for Module Resolution
The updated import statements now explicitly include the .js extension (i.e. ~/runtimes/identifier.js, ~/runtimes/index.js, and ./arguments.js). This change ensures compatibility with ECMAScript module resolution and aligns with the updated build process.

packages/components/package.json (2)

9-10: Updated Export Paths to JSX Files
The export configuration now directs consumers to ./dist/*.jsx for module resolution instead of ./dist/*.js. Ensure that downstream tooling and bundlers are configured to handle JSX outputs accordingly.

---

14-14: Build Script Modification: Switching from tsup to tsc
The "build" script has been updated from using tsup to calling tsc directly. This change meets the PR objective of replacing tsup with the TypeScript compiler. Verify that this provides the expected build outputs and that any necessary TypeScript configurations are in place to handle your project’s requirements.

packages/engine/src/runtimes/fetch.ts (1)

17-18: Consistent Local Module Imports with Explicit Extensions
The updated imports for local modules now include the .js extension (i.e. ./getters.js and ./identifier.js). This change fosters consistency with the updated module resolution strategy across the codebase.

apps/website/tsconfig.json (1)

2-3: React-Specific TypeScript Configuration Update
Updating the "extends" property to "../../tsconfig.react.json" and modifying the "include" array (to include "next-env.d.ts" and .next/types/**/*.ts) aligns this project with a React-specific TypeScript setup. Double-check that these changes integrate smoothly with the surrounding build and linting processes.

packages/engine/src/runtimes/getters.ts (1)

1-4: Enhanced JSON Imports with Explicit Type Declarations
The JSON file imports have been updated to use the with { type: 'json' } clause for RuntimeExamples, RuntimeExtensions, RuntimeIcons, and RuntimeNames. This explicit declaration improves clarity and ensures proper typing under ECMAScript module semantics.

apps/browser-extension/tsconfig.json (1)

2-3: Updated 'extends' and inclusion settings in tsconfig.json.
The configuration now extends from ../../tsconfig.react.json and limits its content to only the necessary "include" property. This streamlined setup suggests that a React-specific base config is now handling previously explicit compiler options. Please verify that no critical customizations (e.g., exclusions or specific compiler options) are lost in transit.

packages/shapes/src/index.ts (1)

1-3: Explicitly specifying '.js' in export statements.
Changing the exports to reference .js files helps align with module resolution requirements (especially given the "typescript.preferences.importModuleSpecifierEnding": "js" setting in your project). Confirm that all consumers of these modules are updated to import using the .js extension.

package.json (3)

6-6: New lint script added using Biome.
The addition of "lint": "biome check . --write" will enforce code style consistency. Ensure that Biome is properly configured across the repository to avoid unexpected linting behaviors.

---

22-22: New development dependency 'ts-patch' added.
Inclusion of "ts-patch": "^3.3.0" aligns with the PR’s objective of replacing tsup, enabling custom TypeScript transformations during installation. Make sure that any build scripts that relied on tsup have been updated accordingly.

---

24-25: Dependency updates: 'typescript' and 'typescript-transform-paths'.
Ensuring that you explicitly depend on TypeScript ^5.7.3 and adding "typescript-transform-paths": "^3.5.5" should support enhanced module path transformations. Confirm that these align with your new tsconfig settings and overall project expectations.

tsconfig.base.json (4)

43-44: Updated 'lib' configuration for ECMAScript compatibility.
Switching the "lib" value to ["ES2022", "DOM", "DOM.Iterable"] harmonizes with the target "ES2022" setting and ensures a more precise set of library features is available during compilation.

---

48-48: Enabled compiler diagnostics.
The addition of "diagnostics": true will aid in revealing underlying compilation issues. Ensure that the increased verbosity does not overwhelm routine builds.

---

51-52: Configured build information and updated path mapping.
Setting "tsBuildInfoFile": "${configDir}/dist/.tsbuildinfo" improves incremental build performance, and updating "paths" to { "~/*": ["src/*"] } removes redundant relative pathing. Verify that these changes correctly reflect the repository’s directory structure.

---

54-57: Added plugins for transforming module paths.
Including the "typescript-transform-paths" plugin—with one instance running post-declarations—ensures that module paths are correctly resolved in both source and declaration outputs. Confirm that this transformation aligns with your intended module resolution strategy.